From f1062ce338370e19109cee0a922baa4398b0c816 Mon Sep 17 00:00:00 2001 From: Corey Farwell Date: Thu, 5 May 2016 21:36:13 -0400 Subject: [PATCH] Utilize `if let` over single branch `match`. --- src/cargo/core/package_id_spec.rs | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/cargo/core/package_id_spec.rs b/src/cargo/core/package_id_spec.rs index a643713b2..031249393 100644 --- a/src/cargo/core/package_id_spec.rs +++ b/src/cargo/core/package_id_spec.rs @@ -17,14 +17,12 @@ pub struct PackageIdSpec { impl PackageIdSpec { pub fn parse(spec: &str) -> CargoResult { if spec.contains("/") { - match spec.to_url() { - Ok(url) => return PackageIdSpec::from_url(url), - Err(..) => {} + if let Ok(url) = spec.to_url() { + return PackageIdSpec::from_url(url); } if !spec.contains("://") { - match Url::parse(&format!("cargo://{}", spec)) { - Ok(url) => return PackageIdSpec::from_url(url), - Err(..) => {} + if let Ok(url) = Url::parse(&format!("cargo://{}", spec)) { + return PackageIdSpec::from_url(url); } } } -- 2.30.2